Skip to content

fix(mcp): make open(path) round-trip and return not_found details - #34

Merged
Razpines merged 1 commit into
mainfrom
fix/mcp-open-path-roundtrip
Feb 28, 2026
Merged

Razpines merged 1 commit into
mainfrom
fix/mcp-open-path-roundtrip

Conversation

@Razpines

Copy link
Copy Markdown
Owner

Summary

  • make DocStore.open_doc(path=...) resolve docs by exact/normalized origin_path
  • add canonical URL lookup support in open_doc(path=...)
  • keep doc-id lookup behavior and add path-derived fallback doc-id normalization
  • change MCP open() missing-doc response to structured
    ot_found payload with attempted inputs
  • add tests covering origin_path round-trip and canonical URL lookup

Issues

Testing

  • python -m py_compile src/unity_docs_mcp/mcp_server.py src/unity_docs_mcp/tools/ops.py tests/test_mcp_meta.py tests/test_ops_open_path_lookup.py
  • $env:PYTHONPATH='C:\projects\UnityRAG\src'; python -m pytest tests/test_ops_open_path_lookup.py -q

@Razpines
Razpines merged commit a86e7b3 into main Feb 28, 2026
4 checks passed
@Razpines
Razpines deleted the fix/mcp-open-path-roundtrip branch February 28, 2026 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bug(mcp): make open(path=origin_path) round-trip from search results

1 participant